Pull Request

🤨 Rationale

Enables the eslint require-await rule after noticing some spots where async was used unexpectedly.

This is disabled in the ni eslint rules due to airbnb's configuration which I think is not well justified. At least in nimble we should prefer async-await syntax AND prefer to not mix async and promise syntax. Enabling this rule helps prevent mixing syntaxes and so far with enabling it helped prevent introducing unnecessary / unexpected async functions.

👩‍💻 Implementation

Enabled require-await or @typescript-eslint/require-await in each .eslintrs.jc file. The rule found additional places in source where the use of async was unnecessary.

🧪 Testing

Rely on CI.

✅ Checklist

• I have updated the project documentation to reflect my changes or determined no changes are needed.